Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Xarray accessor mirroring Raster class #446

Draft
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

rhugonnet
Copy link
Contributor

@rhugonnet rhugonnet commented Jan 24, 2024

Ongoing

TO-DO

  • Add equivalent of Raster.from_array() to RasterAccessor class (or RasterBase class?) and individual setting operations (for transform, crs, nodata, and area_or_point) to make all methods (reproject, etc) naturally work on both Raster and ds.rst,
  • Ensure dual support of masked-array & NaN arrays for all methods (_reproject,, etc),
  • Add delayed function (in _reproject, _interp_points and _subsample) by detecting automatically if input array is delayed.
  • Add tests specific to RasterBase,
  • Add tests specific to RasterAccessor functionalities (comparing to ds.rio).

Other Dask support to add (will be moved as issues for later PRs)

  • The reduce_points function can copy the same logic as interp_points,
  • The crop function using isel of Rioxarray,
  • Potentially look at geocube for rasterize/polygonize support,
  • But the proximity function would be a bit of work...

Resolves #383
Resolves #567

@rhugonnet rhugonnet marked this pull request as draft January 24, 2024 12:34
@rhugonnet rhugonnet changed the title Add Xarray accessor mirroring Raster API Add Xarray accessor mirroring Raster class Jan 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add set_crs and set_transform to force re-set geospatial attributes? Add Xarray accessor gu
1 participant